-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update status of verified configs #198
Conversation
Signed-off-by: Gerwin Klein <[email protected]>
The naming of the i.MX boards is inconsistent, ARM Sabre Lite is The config |
Another comment (not relevant to this PR) is that all the 32-bit ARM configs are named differently depending if hypervisor mode is supported or not but the |
From memory this is a historical thing which probably can't be changed at this point but it's one of the things that makes this page more confusing than it should be (in my opinion). |
It's something we eventually want to get rid of (but currently can't for the old stuff), so we've decided to keep it clean for AARCH64 at least. It does make it inconsistent, but the hope is to resolve that in the future. |
Tbh, I was confused about that as well. The config file for just I guess this means the heading for the other one should be |
Signed-off-by: Gerwin Klein <[email protected]>
183b465
to
d6246ea
Compare
Ok, I've pushed a new version with an attempt at more consistency. The platform/soc/board names are already not very consistent in the main hardware table, esp between the |
arm_hyp could be removed, it's a big change, but doesn't affect verification and so should be tractable. |
We might be talking about different ARM_HYP things -- ARM_HYP is the name of the verification L4V_ARCH setting that selects the Arm 32 bit proofs with hyp. It's also a cmake setting. The cmake setting could be changed without affecting verification, I think, but the name would still be mentioned on this page because the name also encodes proof selection. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this makes sense and looks correct to me.
Thanks for making it less confusing! |
This adds
exynos5
as verified configuration and marks the verification ofimx8mm
as maintained.